misc: Optimize flash and memory usage. #270
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Author
|
FYI @pillo79 @pennam for some reason enabling log level=4 (debug for everything) causes a hardfault on Portenta-H7. It shouldn't be a stack overflow issue, but I tried increasing all stacks anyway and it doesn't help. That said, it is very convenient and I have been using it locally, instead of adding debug options manually to conf files, so I will get it fixed somehow and let you know. |
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
0df6fb2 to
46d57bd
Compare
Author
|
Actually, it turns out that it's just some log thread overflow, fixable by increasing stack sizes. However,, I decided to Not enable debug logs globally, because we don't really have extra memory for that and it makes it much harder to see the debug log you're looking for. |
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
46d57bd to
0cf7454
Compare
Add debug config fragment to easily build debug images. Usage: ./extra/build.sh <board> --debug The build includes extra/debug.conf which enables: - Debug mode with assertions and verbose output - Debug-level logging (level 4) - Stack overflow detection - Thread monitoring - Debug optimizations (-Og) - Enable shell. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
0cf7454 to
ccd08d2
Compare
iabdalkader
changed the title
The created sketch log structures depends on the config: CONFIG_LOG (IMMEDIATE/DEFERRED): creates log_const_sketch struct. CONFIG_LOG_RUNTIME_FILTERING (implied by CONFIG_SHELL): creates log_dynamic_sketch struct. CONFIG_LOG_MODE_IMMEDIATE + CONFIG_SHELL: creates both. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Replace the libc's malloc, free etc.. with wrappers that redirect memory allocation Zephyr's kernel heap functions (k_malloc/k_free). Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
This saves about 70KBs FLASH and ~47KBs RAM. The saved memory is used to increase mbedtls heap to support cert loading, and the remaining added to main heap. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
ccd08d2 to
3fed70a
Compare
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
3fed70a to
5d1e8f5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR optimizes flash and memory usage, saving up to ~80KBs flash and ~50KBs RAM on some boards, by disabling the shell for production builds, CPP and using a unified heap for Zephyr and sketches. The saved memory is used to increase mbedtls heap to support cert loading, and the remaining added to main heap. For Portenta, ~100KBs should be available to user sketches.
Changes:
To build debug images use:
./extra/build.sh <board> --debugThe build includes extra/debug.conf which enables: